Skip to content

Fix globals' type deduction in TypeScavenger#2431

Merged
MrSidims merged 1 commit intoKhronosGroup:mainfrom
mshelego:opaque_getelementrptr
Mar 18, 2024
Merged

Fix globals' type deduction in TypeScavenger#2431
MrSidims merged 1 commit intoKhronosGroup:mainfrom
mshelego:opaque_getelementrptr

Conversation

@mshelego
Copy link
Contributor

If a global is initialized with a ConstantExpr returning a pointer (getelementptr), the right type has to be deduced instead of defaulting it to i8*.

@CLAassistant
Copy link

CLAassistant commented Mar 12, 2024

CLA assistant check
All committers have signed the CLA.

@mshelego mshelego force-pushed the opaque_getelementrptr branch from 7e2fad8 to cee3358 Compare March 12, 2024 11:19
@mshelego mshelego closed this Mar 12, 2024
@mshelego mshelego reopened this Mar 12, 2024
If a global is initialized with a ConstantExpr returning a pointer
(getelementptr), the right type has to be deduced instead of defaulting
it to i8*.
@mshelego mshelego force-pushed the opaque_getelementrptr branch from cee3358 to 3434dca Compare March 18, 2024 11:02
Copy link
Contributor

@MrSidims MrSidims left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@MrSidims MrSidims merged commit a7a36de into KhronosGroup:main Mar 18, 2024
@wenju-he
Copy link
Contributor

wenju-he commented Mar 3, 2026

/backport llvm_release_180

@github-actions
Copy link

github-actions bot commented Mar 3, 2026

Attempting to create backport to llvm_release_180...

github-actions bot pushed a commit that referenced this pull request Mar 3, 2026
If a global is initialized with a ConstantExpr returning a pointer
(getelementptr), the right type has to be deduced instead of defaulting
it to i8*.
@github-actions
Copy link

github-actions bot commented Mar 3, 2026

Success. Backport PR created: #3613

vmaksimo pushed a commit that referenced this pull request Mar 9, 2026
…venger (#3613)

Backport of PR #2431 into `llvm_release_180`.

All commits applied cleanly.

Authored-by: Maksim Shelegov <maksim.shelegov@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants